Skip to content

feat: early return if the l1 fee scalar is zero#3213

Merged
rakita merged 2 commits intobluealloy:mainfrom
datnguyencse:feat-early-return-if-zero-l1-scalar
Dec 8, 2025
Merged

feat: early return if the l1 fee scalar is zero#3213
rakita merged 2 commits intobluealloy:mainfrom
datnguyencse:feat-early-return-if-zero-l1-scalar

Conversation

@datnguyencse
Copy link
Contributor

On certain chains (ours included), the L1 fee scalar is set to zero to simplify the fee formula. It would be beneficial to return early during L1 fee calculation when the scalar is zero, thereby avoiding the expensive transaction-size estimation.

Thanks and I’d appreciate your review, and any comments or suggestions are very welcome.

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 8, 2025

CodSpeed Performance Report

Merging #3213 will not alter performance

Comparing datnguyencse:feat-early-return-if-zero-l1-scalar (b4ec665) with main (eb37a15)

Summary

✅ 173 untouched

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, i see logic is not changed

@rakita rakita merged commit 99e6f34 into bluealloy:main Dec 8, 2025
31 checks passed
@github-actions github-actions bot mentioned this pull request Dec 8, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
* feat: early return if the l1 fee scalar is zero

* update code format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants